Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

precompile: Remove extra checks #1538

Merged
merged 8 commits into from
Sep 12, 2023
Merged

Conversation

mustermeiszer
Copy link
Collaborator

@mustermeiszer mustermeiszer commented Sep 8, 2023

Description

Dropping any EVM storage related validation that we have in place and treat this precompile as a simple forwarded for gateway messages.

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@cdamian cdamian force-pushed the feat/drop-axelar-precompile-checks branch from 16eb0b0 to 27adf18 Compare September 8, 2023 21:51
@cdamian cdamian force-pushed the feat/drop-axelar-precompile-checks branch 2 times, most recently from bb617b8 to 21717c3 Compare September 10, 2023 19:20
@cdamian cdamian marked this pull request as ready for review September 10, 2023 20:00
@cdamian cdamian force-pushed the feat/drop-axelar-precompile-checks branch from 55551ec to 31aa21e Compare September 10, 2023 22:08
@cdamian cdamian force-pushed the feat/drop-axelar-precompile-checks branch 2 times, most recently from 5c0da30 to d0329d2 Compare September 11, 2023 15:19
@mustermeiszer
Copy link
Collaborator Author

I can not approve as I created the PR. But looks all good to me!

@cdamian cdamian force-pushed the feat/drop-axelar-precompile-checks branch from d0329d2 to a46c234 Compare September 12, 2023 15:44
@cdamian cdamian merged commit c779524 into main Sep 12, 2023
10 checks passed
@cdamian cdamian deleted the feat/drop-axelar-precompile-checks branch September 12, 2023 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants